Skip to content

Comments

ci: add auto-approval for copilot reviews#4

Merged
troinine merged 9 commits intomainfrom
chore/pr-approval
Dec 9, 2025
Merged

ci: add auto-approval for copilot reviews#4
troinine merged 9 commits intomainfrom
chore/pr-approval

Conversation

@troinine
Copy link
Owner

@troinine troinine commented Dec 9, 2025

No description provided.

@troinine troinine added the ignore-for-release Ignored from release notes label Dec 9, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a GitHub Actions workflow that automatically approves pull requests when GitHub Copilot reviews them with no new comments, intended to streamline the review process for clean PRs.

Key Changes:

  • Introduces workflow triggered by PR review events to detect clean Copilot reviews
  • Implements safeguards to prevent auto-approval of PRs modifying sensitive files (workflows)
  • Adds approval automation via GitHub CLI when conditions are met

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI commented Dec 9, 2025

@troinine I've opened a new pull request, #5, to work on those changes. Once the pull request is ready, I'll request review from you.

troinine and others added 5 commits December 9, 2025 21:07
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Tero Roininen <troinine@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Tero Roininen <troinine@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Tero Roininen <troinine@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Tero Roininen <troinine@users.noreply.github.com>
* Initial plan

* fix: properly escape dots in sensitive file pattern matching

Co-authored-by: troinine <13124764+troinine@users.noreply.github.com>

* Merge branch 'chore/pr-approval' into copilot/sub-pr-4

Co-authored-by: troinine <13124764+troinine@users.noreply.github.com>

---------

Signed-off-by: Tero Roininen <troinine@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: troinine <13124764+troinine@users.noreply.github.com>
Co-authored-by: Tero Roininen <troinine@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

troinine and others added 2 commits December 9, 2025 21:31
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Tero Roininen <troinine@users.noreply.github.com>
@troinine troinine merged commit 8cf801f into main Dec 9, 2025
3 checks passed
@troinine troinine deleted the chore/pr-approval branch December 9, 2025 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore-for-release Ignored from release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants